Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Placemark New Features #1263

Merged
merged 19 commits into from
Nov 21, 2024
Merged

Conversation

nickcastel50
Copy link
Contributor

@nickcastel50 nickcastel50 commented Oct 31, 2024

Refactored and implemented New Placemarks Version

Features:

  • Sprite based placemark system, placemarks viewable at all times in the model if notes are open
  • Add placemark to issue or comment
  • Persistence in Github
  • Selecting a placemark scrolls notes view to the linked issue or placemark

Testing

Select the different markers to see the URL change and current issue / comment note update in the UI.

To place a placemark, click into an issue thread. If you are logged in, you will see a placemark Icon on the main issue and on the "leave a comment" UI elements. To leave one on an existing issue, go to edit issue and then click the marker icon for the issue. To leave one on a comment, click the marker icon on the new comment UI element. Then, double click anywhere on the model.

@nickcastel50 nickcastel50 added the enhancement New feature or request label Oct 31, 2024
@nickcastel50 nickcastel50 self-assigned this Oct 31, 2024
Copy link

netlify bot commented Oct 31, 2024

Deploy Preview for bldrs-share ready!

Name Link
🔨 Latest commit 2f812b7
🔍 Latest deploy log https://app.netlify.com/sites/bldrs-share/deploys/673f9d10b93b5a00085acdbe
😎 Deploy Preview https://deploy-preview-1263--bldrs-share.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 17 (🟢 up 1 from production)
Accessibility: 89 (no change from production)
Best Practices: 92 (no change from production)
SEO: 100 (no change from production)
PWA: -
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

netlify bot commented Oct 31, 2024

Deploy Preview for bldrs-share-prod ready!

Name Link
🔨 Latest commit 2f812b7
🔍 Latest deploy log https://app.netlify.com/sites/bldrs-share-prod/deploys/673f9d10c0828b0007003725
😎 Deploy Preview https://deploy-preview-1263--bldrs-share-prod.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

…exists there. * set up state for editable issues + edit bodies for proper placemark insertion * add cancel edit button to leave editing state in notes
@nickcastel50 nickcastel50 marked this pull request as ready for review November 20, 2024 21:45
src/Containers/CadView.jsx Outdated Show resolved Hide resolved
src/hooks/usePlaceMark.js Outdated Show resolved Hide resolved
src/Infrastructure/PlaceMark.js Show resolved Hide resolved
src/Components/Notes/hashState.js Outdated Show resolved Hide resolved
src/Components/Notes/NoteCardCreate.jsx Show resolved Hide resolved
src/Infrastructure/PlaceMark.js Show resolved Hide resolved
src/hooks/usePlaceMark.js Outdated Show resolved Hide resolved
src/hooks/usePlaceMark.js Outdated Show resolved Hide resolved
src/hooks/usePlaceMark.js Outdated Show resolved Hide resolved
src/hooks/usePlaceMark.js Outdated Show resolved Hide resolved
@nickcastel50 nickcastel50 merged commit 1591a80 into bldrs-ai:main Nov 21, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants